Skip to content

Allow overriding TUTORIALS_REF in generate_reference_results.py#869

Merged
MakisH merged 7 commits into
precice:developfrom
PranjalManhgaye:issue-844-clean
Jul 9, 2026
Merged

Allow overriding TUTORIALS_REF in generate_reference_results.py#869
MakisH merged 7 commits into
precice:developfrom
PranjalManhgaye:issue-844-clean

Conversation

@PranjalManhgaye

@PranjalManhgaye PranjalManhgaye commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

When adding a new tutorial on a feature branch, generating reference results used to require temporarily editing reference_versions.yaml to set TUTORIALS_REF , then changing it back. That was easy to forget and messy in PRs.

This PR adds a simpler way:

  • --build_args on generate_reference_results.py (same idea as systemtests.py)
  • Optional build_args input on the generate-reference-results GitHub Actions workflow
  • Use tutorials from the dropdown: reference_versions.yaml (default) or workflow branch (TUTORIALS_REF:<from_ref>). build_args overrides if set.

Local Testing

Closes precice#844. Adds --build_args to the reference-generation script and
workflow inputs; defaults TUTORIALS_REF to from_ref when not specified.

@MakisH MakisH left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already very nice, thanks!

Testing works: https://github.com/precice/tutorials/actions/runs/29015604369

Just one point regarding the GHA workflow inputs.

Comment thread tools/tests/generate_reference_results.py
Comment thread .github/workflows/generate-reference-results.yml Outdated
@PranjalManhgaye PranjalManhgaye requested a review from MakisH July 9, 2026 16:50

@MakisH MakisH left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some tiny updates, and then ready to merge.

Comment thread tools/tests/README.md Outdated
Comment thread .github/workflows/generate-reference-results-manual.yml Outdated
@PranjalManhgaye PranjalManhgaye requested a review from MakisH July 9, 2026 19:00
Comment thread tools/tests/README.md
@MakisH MakisH merged commit 60c2a22 into precice:develop Jul 9, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Needs review to Done in GSoC 2026: System tests improvements Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

Overriding the TUTORIALS_REF used by generate_reference_results.py

2 participants